Add Quine trait to AST, make pretty public.
#416
Conversation
|
ha. the annotation fix is already in. didn't realize that. |
| use std::fmt; | ||
|
|
||
| #[cfg(feature = "quine")] | ||
| use {polyquine::Quine, proc_macro2::TokenStream}; |
There was a problem hiding this comment.
Quine is a pretty old package - is it maintained?
There was a problem hiding this comment.
it's polyquine, not quine. it's actually maintained. I've contributed a couple of patches to make this bit of code work
#[derive(Debug, Eq, PartialEq)]
#[cfg_attr(feature = "quine", derive(Quine), path_prefix(typeql::query::schema))]
pub struct Define {
pub span: Option<Span>,
pub definables: Vec<Definable>,
}
|
Sorry i've just had a lot come up! I need to chat with the team what we should do with these specialized implementations that are gated behind feature flags! |
|
Hey @flyingsilverfin no worries. I got a bit stuck with this entire ordeal too due to macro site hygiene issues and haven't been able to figure out how to mix schemas and queries. I was thinking about two approaches, both with their own downsides.
Ideally I would love to have a solution that combines both but I'm not there yet. I'll keep you updated with more information as I keep working on this. Already responding to this has helped me figure some things out 😄 Hope you have a wonderful week |
2 participants
Product change and motivation
prettypublic.derive(Quine)to AST members.Implementation
Adds an optional quine derive feature to the package that will implement Quine
for the AST to make it possible for the ASTs to reproduce them selves.